Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add breaking changes entry #7900 #7928

Merged
merged 3 commits into from
Oct 12, 2023
Merged

Add breaking changes entry #7900 #7928

merged 3 commits into from
Oct 12, 2023

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Oct 12, 2023

No description provided.

@zanieb zanieb changed the title Add breaking changes entry for https://github.com/astral-sh/ruff/pull/7900 Add breaking changes entry #7900 Oct 12, 2023
@zanieb zanieb force-pushed the zanie/breaking-ruleset branch from 4fe00ca to ef5dd81 Compare October 12, 2023 15:01
Pycodestyle prefixes `E4`, `E7`, and `E9` to exclude rules that conflict with automatic formatters. Consequently,
the stable rule set no longer includes `line-too-long` (`E501`) and `mixed-spaces-and-tabs` (`E101`). Other
excluded Pycodestyle rules include whitespace enforcement in `E1` and `E2` and are marked as unstable and
not on by default already. These rules may still be enabled and if `select` has been configured there will be no change.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tweaked the last two sentences; feel free to take or leave:

Previously, Ruff enabled all implemented rules in Pycodestyle (`E`) by default. Ruff now only includes the
Pycodestyle prefixes `E4`, `E7`, and `E9` to exclude rules that conflict with automatic formatters. Consequently,
the stable rule set no longer includes `line-too-long` (`E501`) and `mixed-spaces-and-tabs` (`E101`). Other
excluded Pycodestyle rules include whitespace enforcement in `E1` and `E2`; these rules are currently in preview,
and so are already omitted by default.

This change only affects those using Ruff under its default rule set. Users that include `E` in their `select` will
experience no change in behavior.

@zanieb zanieb enabled auto-merge (squash) October 12, 2023 15:50
@zanieb zanieb merged commit 48b256b into main Oct 12, 2023
@zanieb zanieb deleted the zanie/breaking-ruleset branch October 12, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants